Skip to content

ci: fix failing Differential ShellCheck workflow - #1121

Closed
jamacku wants to merge 1 commit into
nodenv:mainfrom
jamacku:fix-fatal-error
Closed

ci: fix failing Differential ShellCheck workflow#1121
jamacku wants to merge 1 commit into
nodenv:mainfrom
jamacku:fix-fatal-error

Conversation

@jamacku

@jamacku jamacku commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Differential ShellCheck requires a specified branch in the workflow file. To avoid this error:

fatal: Invalid revision range e7d05fd3533403e8aa79d4c22b31d106ec3877ac..cdb538f5c530addbc82c353a94e5d29844d043d4

This should resolve the issue described in:

Differential ShellCheck requires a specified branch in the workflow file. To avoid this error:

```
fatal: Invalid revision range e7d05fd..cdb538f
```
name: Test
on:
push:
branches: [ main, v5 ]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't seem to make sense to me. I would expect differential shellcheck to be run on any branch that's pushed, not just two arbitrary configured branches. That's why the branches weren't limited in the push events before.

@jasonkarns

Copy link
Copy Markdown
Member

Closing this. differential shellcheck should run on any PR, not just specific base branches. Similarly with pushes. There's no reason it should be limited to an arbitrary set of branches.

@jasonkarns jasonkarns closed this Jul 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants